-
-
Notifications
You must be signed in to change notification settings - Fork 33.2k
Add documentation for remote debugging with pdb #134260
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
CC @pablogsal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, -m pid is not documented at https://docs.python.org/dev/library/pdb.html. You may update this option there as well.
|
Yeah, maybe update the doc file as well. |
d4e233e to
f31f786
Compare
f31f786 to
d2db5b9
Compare
Adds a brief note to the pdb help summary about attaching to a running process using the -p option, making the remote debugging feature more visible.
23f6521 to
cbf115c
Compare
|
I think normally we have a bot that gives a preview of the docs when we changes the docs, any idea what happened @vstinner ? |
You can click on the "readthedocs" CI to go to the preview doc: https://cpython-previews--134260.org.readthedocs.build/en/134260/library/pdb.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for updating pdb doc as well.
|
Thanks @ivonastojanovic for the PR, and @pablogsal for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14. |
* Mention remote debugging via -p PID in usage text Adds a brief note to the pdb help summary about attaching to a running process using the -p option, making the remote debugging feature more visible. * Mention remote debugging in pdb.rst (cherry picked from commit 6856a04) Co-authored-by: ivonastojanovic <[email protected]>
|
GH-134398 is a backport of this pull request to the 3.14 branch. |
|
I think the part where it explains how to use attach, should be part of the |
* Mention remote debugging via -p PID in usage text Adds a brief note to the pdb help summary about attaching to a running process using the -p option, making the remote debugging feature more visible. * Mention remote debugging in pdb.rst
* Mention remote debugging via -p PID in usage text Adds a brief note to the pdb help summary about attaching to a running process using the -p option, making the remote debugging feature more visible. * Mention remote debugging in pdb.rst
* Mention remote debugging via -p PID in usage text Adds a brief note to the pdb help summary about attaching to a running process using the -p option, making the remote debugging feature more visible. * Mention remote debugging in pdb.rst
Adds a brief note to the pdb help summary about attaching to a running process using the -p option, making the remote debugging feature more visible.